Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix post deployment hook for /etc/hosts in multi-stack #985

Conversation

bogdando
Copy link
Contributor

@bogdando bogdando commented Dec 19, 2024

Add /etc/hosts entries for all cells' nodes on internal_api network as well. Enable rabbitmq connections from superconductor and scheduler via transport URLs defined for other cells'.

NOTE: tripleo_stack_name only used in start/end markers by the tripleo_hosts_entries role. Use a non-conflicting "stack" name for markers to append to hosts entries created for the overcloud stack, and stacks created for additional cells.

Required-by: openstack-k8s-operators/data-plane-adoption#746

@bogdando
Copy link
Contributor Author

bogdando commented Dec 19, 2024

no need for test project, multi-cell tripleo code is still noop (only 1 cell is deployed in adoption CI yet)

Add /etc/hosts entries for all cells' nodes on internal_api network
as well. Enable rabbitmq connections from superconductor and
scheduler via transport URLs defined for other cells'.

NOTE: tripleo_stack_name only used in start/end markers by the
tripleo_hosts_entries role. Use a non-conflicting "stack" name
for markers to append to hosts entries created for the overcloud
stack, and stacks created for additional cells.

Signed-off-by: Bohdan Dobrelia <[email protected]>
@bogdando bogdando force-pushed the tripleo_multi_stack branch from 295084a to 76ae8c5 Compare December 20, 2024 10:13
@fao89
Copy link
Contributor

fao89 commented Jan 6, 2025

/approve

@openshift-ci openshift-ci bot added the approved label Jan 6, 2025
@stuggi stuggi added the lgtm label Jan 9, 2025
Copy link
Contributor

openshift-ci bot commented Jan 9, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: bogdando, fao89, slagle

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@bogdando
Copy link
Contributor Author

bogdando commented Jan 9, 2025

Happy New Year, folks!

Can we have this merged, please? @fao89 @jistr @stuggi @olliewalsh

@stuggi
Copy link
Contributor

stuggi commented Jan 9, 2025

Happy New Year, folks!

Can we have this merged, please? @fao89 @jistr @stuggi @olliewalsh

its already in merge pool

@openshift-merge-bot openshift-merge-bot bot merged commit b989130 into openstack-k8s-operators:main Jan 9, 2025
5 checks passed
@bogdando bogdando deleted the tripleo_multi_stack branch January 10, 2025 12:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants